-
Notifications
You must be signed in to change notification settings - Fork 398
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use HeadBucket instead of GetBucketLocation (#1979) #1987
base: main
Are you sure you want to change the base?
Use HeadBucket instead of GetBucketLocation (#1979) #1987
Conversation
Build succeeded. ❌ ansible-galaxy-importer FAILURE in 5m 27s (non-voting) |
I'm facing the same error reported on #1979 in this exact scenario: |
Would like to see this merged as well. Facing the exact error too. |
What needs to be done in order to get this PR merged? I'd love to see this merged as well. |
I have the same problem with this plugin and GetBucketLocation and it is blocking me from doing production deployments in a multi-region environment using Ansible and SSM. When might this PR be reviewed/merged? |
recheck |
Merge Failed. This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. |
ba06771
to
c9a3758
Compare
Build succeeded. ❌ ansible-galaxy-importer FAILURE in 5m 30s (non-voting) |
Hi. I'm also affected in my job production environment. Is there any major problem? Thanks. |
Currently we need a 2nd review. |
this is also blocking us. looking foward to see it merged |
regate |
regate |
Build failed (gate pipeline). For information on how to proceed, see https://ansible.softwarefactory-project.io/zuul/buildset/8c1a5f00231d4290b41f705a8eabc906 ❌ ansible-galaxy-importer FAILURE in 4m 38s (non-voting) |
regate |
Build failed (gate pipeline). For information on how to proceed, see https://ansible.softwarefactory-project.io/zuul/buildset/71748f80e37541efb5c7238fe1edee4f ✔️ ansible-galaxy-importer SUCCESS in 8m 54s (non-voting) |
regate |
Build failed (gate pipeline). For information on how to proceed, see https://ansible.softwarefactory-project.io/zuul/buildset/508bc8621a3d4e71ab5ec9576557cf99 ❌ ansible-galaxy-importer FAILURE in 4m 35s (non-voting) |
regate |
Build failed (gate pipeline). For information on how to proceed, see https://ansible.softwarefactory-project.io/zuul/buildset/d8b93e8d11d041bca0f8a524e815001f ❌ ansible-galaxy-importer FAILURE in 4m 34s (non-voting) |
Hi. Is there any problem or something that we could do to move forward and merge this? We are in a bit of a hurry because of our business deployments and deadlines. We don't wanna sound rude or disrespectful. Thank you for your time and effort. |
regate |
Build failed (gate pipeline). For information on how to proceed, see https://ansible.softwarefactory-project.io/zuul/buildset/fb892042c55446cb8fa551df19bd72ae ✔️ ansible-galaxy-importer SUCCESS in 3m 06s (non-voting) |
I've no idea. Someone from the RedHat Team must take a look @alinabuzachis @hakbailey @tremble |
regate |
Build failed (gate pipeline). For information on how to proceed, see https://ansible.softwarefactory-project.io/zuul/buildset/e6860e3fff514e788a3e8471d0bce934 ❌ ansible-galaxy-importer FAILURE in 4m 45s (non-voting) |
Build failed (gate pipeline). For information on how to proceed, see https://ansible.softwarefactory-project.io/zuul/buildset/be534b03afb54007aa4302fcf1d35fd9 ✔️ ansible-galaxy-importer SUCCESS in 3m 23s (non-voting) |
regate |
Build failed (gate pipeline). For information on how to proceed, see https://ansible.softwarefactory-project.io/zuul/buildset/7decb0a13e6642d3bf002b9d4aa5a9ef ❌ ansible-galaxy-importer FAILURE in 5m 40s (non-voting) |
Greetings, Is there any update for this? We have several production ops blocked because of this. Thanks. |
recheck |
Build failed. ✔️ ansible-galaxy-importer SUCCESS in 3m 13s (non-voting) |
recheck |
Build failed (gate pipeline). For information on how to proceed, see https://ansible.softwarefactory-project.io/zuul/buildset/79514486e3004b93b6e3c572f22beb67 ✔️ ansible-galaxy-importer SUCCESS in 14m 29s (non-voting) |
Build failed. ✔️ ansible-galaxy-importer SUCCESS in 3m 39s (non-voting) |
recheck |
Build failed. ❌ ansible-galaxy-importer FAILURE in 4m 26s (non-voting) |
SUMMARY
Replacing the call to get_bucket_location with a call to head_bucket in Connection._get_bucket_endpoint().
The GetBucketLocation API call only works from the bucket owner account. This enables using a bucket owned by another accout, e.g. a shared organization bucket when running cross-account.
Fixes #1979.
ISSUE TYPE
COMPONENT NAME
aws_ssm
ADDITIONAL INFORMATION
The official documentation for the GetBucketLocation API call states it is only supported for backwards compatibility and recomends using HeadBucket instead.